-
Notifications
You must be signed in to change notification settings - Fork 118
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
부산대 BE_문성재 2주차 과제 (2단계) #294
Open
student-p
wants to merge
24
commits into
kakao-tech-campus-2nd-step2:student-p
Choose a base branch
from
student-p:step2
base: student-p
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+909
−1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
구현할 기능 정의
id설정부분을 사용자가 접근하지 못하도록 수정
상품 수정 메서드에서 PutMapping으로 수정 상품 삭제 메서드에서 DeleteMapping으로 수정
step2구현을 위한 필요 기능 정의
form데이터가 정상적으로 받아지지 않아서 record에서 일반 클래스로 변경
json을 반환하지 않고 데이터를 html에 랜더링 해서 반환한다 상품의 조회,추가,수정,삭제 기능도 화면상에서 form데이터를 제출하면 가능하도록 변경
step3 구현에 필요한 요구사항 정의 기존에 저장하던 hashMap에서 h2데이터베이스로 저장소를 변경한다. 이를 위해 jdbcTemplate를 이용한다.
cost값이 음수로 되지 않도록 수정
기존 hsahMap에서 데이터베이스로 저장소 변경
step1을 위한 요구사항 작성
유효성 검사를 위한 사용자 정의 예외를 만들고 GlobalExceptionHandler로 전역에서 잡아서 에러를 처리하도록 구현
step2 기능 요구사항 작성
정상적으로 들어왔을 때 올바른 응답이 나가도록 구현
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
시간이 부족해서 예외처리를 못 한 부분이 많습니다
현재 토큰을 email 앞에 언더바를 붙여서 저장하고 있는데 이것도 웹토큰 사용하는 방법을 공부해서 수정해 보겠습니다
현재 UserLogin내부에서 Request와 Response를 static으로 선언해서 UserLogin.Request 이런 식으로 사용하고 있는데 이렇게 사용해도 괜찮은가요? 예상되는 문제가 있다면 뭐가 있을지 궁금합니다.